Skip to content

OBSDOCS-974 - Logging 5.7.13 Release Notes #75244

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
May 2, 2024
Merged

OBSDOCS-974 - Logging 5.7.13 Release Notes #75244

merged 1 commit into from
May 2, 2024

Conversation

shreyasiddhartha
Copy link
Contributor

@shreyasiddhartha shreyasiddhartha commented Apr 29, 2024 •
edited
Loading

@openshift-ci openshift-ci bot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Apr 29, 2024
@ocpdocs-previewbot
Copy link

ocpdocs-previewbot commented Apr 29, 2024 •
edited
Loading

🤖 Thu May 02 11:18:14 - Prow CI generated the docs preview:

https://75244--ocpdocs-pr.netlify.app/
https://75244--ocpdocs-pr.netlify.app/openshift-enterprise/latest/observability/logging/logging_release_notes/logging-5-7-release-notes.html

@anpingli
Copy link

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Apr 29, 2024
periklis
periklis suggested changes Apr 30, 2024
View reviewed changes
Copy link

@periklis periklis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing bug fixes: LOG-5393, LOG-5403

@periklis
Copy link

/ohld

@periklis
Copy link

/hold

@openshift-ci openshift-ci bot added do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. and removed lgtm Indicates that a PR is ready to be merged. labels Apr 30, 2024
@periklis
Copy link

/hold cancel

@periklis
Copy link

/lgtm

@openshift-ci openshift-ci bot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Apr 30, 2024
@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Apr 30, 2024
@shreyasiddhartha
Copy link
Contributor Author

/label peer-review-needed

@openshift-ci openshift-ci bot added the peer-review-needed Signifies that the peer review team needs to review this PR label Apr 30, 2024
@kabirbhartiRH
Copy link

@shreyasiddhartha FYI, LOG-5403 and LOG-5393 are enhancements and not bugs.

@mburke5678 mburke5678 added the peer-review-in-progress Signifies that the peer review team is reviewing this PR label Apr 30, 2024
mburke5678
mburke5678 reviewed Apr 30, 2024
View reviewed changes
@mburke5678 mburke5678 added peer-review-done Signifies that the peer review team has reviewed this PR and removed peer-review-in-progress Signifies that the peer review team is reviewing this PR peer-review-needed Signifies that the peer review team needs to review this PR labels Apr 30, 2024
@mburke5678
Copy link
Contributor

mburke5678 commented Apr 30, 2024 •
edited by openshift-ci bot
Loading

@shreyasiddhartha One comment. Otherwise, pending the issue with LOG-5403 and LOG-5393, LGTM.

@openshift-ci openshift-ci bot removed the lgtm Indicates that a PR is ready to be merged. label May 2, 2024
@openshift-ci OpenShift CI
Copy link

openshift-ci bot commented May 2, 2024

New changes are detected. LGTM label has been removed.

@shreyasiddhartha shreyasiddhartha force-pushed the logging-5.7.13-rn-4.14 branch from c9c0adb to a4d031b Compare May 2, 2024 11:08
@shreyasiddhartha
Copy link
Contributor Author

/label merge-review-needed

@openshift-ci OpenShift CI
Copy link

openshift-ci bot commented May 2, 2024

@shreyasiddhartha: all tests passed!

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@openshift-ci openshift-ci bot added the merge-review-needed Signifies that the merge review team needs to review this PR label May 2, 2024
@mburke5678 mburke5678 merged commit f6ec8e5 into openshift:enterprise-4.14 May 2, 2024
@mburke5678
Copy link
Contributor

/cherrypick enterprise-4.12

@mburke5678
Copy link
Contributor

/cherrypick enterprise-4.13

@openshift-cherrypick-robot
Copy link

@mburke5678: new pull request created: #75479

In response to this:

/cherrypick enterprise-4.12

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

This was referenced May 2, 2024
Merged
Merged
@openshift-cherrypick-robot
Copy link

@mburke5678: new pull request created: #75480

In response to this:

/cherrypick enterprise-4.13

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mburke5678 mburke5678 mburke5678 left review comments

+1 more reviewer

@periklis periklis periklis requested changes

Reviewers whose approvals may not affect merge requirements
Assignees

@periklis periklis

@anpingli anpingli

Labels
merge-review-needed Signifies that the merge review team needs to review this PR peer-review-done Signifies that the peer review team has reviewed this PR size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@shreyasiddhartha @ocpdocs-previewbot @anpingli @periklis @kabirbhartiRH @mburke5678 @openshift-cherrypick-robot